Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v17] Adjust levels of Vale rules #49661

Open
wants to merge 1 commit into
base: branch/v17
Choose a base branch
from

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Dec 2, 2024

Backports #48850

Change the error levels of Vale style rules to pave the way for requiring the Vale linter. Use the warning level for all rules except for the step numbering rule. This mostly preserves the existing "check prose style" GitHub Actions job behavior, in which Vale style rules are optional. The one rule we require is to ensure that numbered Step n/d headings follow the correct numbering scheme.

Copy link

github-actions bot commented Dec 2, 2024

🤖 Vercel preview here: https://docs-djejmanvg-goteleport.vercel.app/docs

@ptgott
Copy link
Contributor Author

ptgott commented Dec 4, 2024

@mmcallister Following up on this one if you have time. Thanks!

@ptgott ptgott added the no-changelog Indicates that a PR does not require a changelog entry label Dec 4, 2024
@ptgott ptgott enabled auto-merge December 4, 2024 21:36
Backports #48850

Change the error levels of Vale style rules to pave the way for
requiring the Vale linter. Use the `warning` level for all rules except
for the step numbering rule. This mostly preserves the existing "check
prose style" GitHub Actions job behavior, in which Vale style rules are
optional. The one rule we require is to ensure that numbered `Step n/d`
headings follow the correct numbering scheme.
Copy link

github-actions bot commented Dec 4, 2024

🤖 Vercel preview here: https://docs-jf25mpuep-goteleport.vercel.app/docs

@ptgott ptgott added this pull request to the merge queue Dec 4, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport documentation no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants